Skip to content

feat: add Codecov integration with cargo-llvm-cov#54

Closed
ptomanRH wants to merge 2 commits into
guacsec:mainfrom
ptomanRH:add-codecov-coverage
Closed

feat: add Codecov integration with cargo-llvm-cov#54
ptomanRH wants to merge 2 commits into
guacsec:mainfrom
ptomanRH:add-codecov-coverage

Conversation

@ptomanRH

@ptomanRH ptomanRH commented Jun 2, 2026

Copy link
Copy Markdown

Summary

  • Add cargo-llvm-cov for Rust coverage report generation (same tooling as main trustify repo)
  • Add push-to-main trigger to CI workflow (previously only ran on PRs — needed for Codecov baseline)
  • Replace cargo test with cargo llvm-cov --codecov to generate Codecov-format output
  • Add Codecov upload step using OIDC authentication (no secret needed)
  • Add codecov.yml with informational status checks, unit-tests flag with carryforward

Changes

File Change
.github/workflows/_main.yaml Add push: branches: [main] trigger, permissions: id-token: write, install cargo-llvm-cov, replace test step with coverage, add Codecov upload
codecov.yml Coverage config: informational checks, flag setup, ignore pkg/config/

Manual follow-up

After merging:

  1. Go to https://app.codecov.io/gh/guacsec/trustify-release-tools
  2. Click the Flags tab → Enable flag analytics

Ref: COVERPORT-257

- Add cargo-llvm-cov for coverage report generation
- Add push-to-main trigger (needed for Codecov baseline)
- Add Codecov upload step with OIDC auth and unit-tests flag
- Add codecov.yml with informational status checks and carryforward

Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
@codecov-commenter

Copy link
Copy Markdown

Welcome to Codecov 🎉

Once you merge this PR into your default branch, you're all set! Codecov will compare coverage reports and display results in all future pull requests.

Thanks for integrating Codecov - We've got you covered ☂️

Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
@ptomanRH

Copy link
Copy Markdown
Author

code removed, no longer needed

@ptomanRH ptomanRH closed this Jun 11, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants